Skip to content
This repository was archived by the owner on Apr 12, 2021. It is now read-only.

wip: verify script#245

Draft
tynes wants to merge 1 commit intomasterfrom
feat/verify-2
Draft

wip: verify script#245
tynes wants to merge 1 commit intomasterfrom
feat/verify-2

Conversation

@tynes
Copy link
Copy Markdown
Collaborator

@tynes tynes commented Feb 24, 2021

Description

WIP script to verify all of the contracts

Metadata

Fixes

  • Fixes # [Link to Issue]

Contributing Agreement

@tynes
Copy link
Copy Markdown
Collaborator Author

tynes commented Mar 1, 2021

This script fails to work for the kovan deployed OVM_L1ETHGateway. Will need an alternative means to verify the contract

@ben-chain
Copy link
Copy Markdown
Collaborator

@tynes FYI there is a good chance #254 requires new genesis for Kovan anyway; maybe we will have more luck verifying the newer contract.

@tynes
Copy link
Copy Markdown
Collaborator Author

tynes commented Mar 2, 2021

@ben-chain Ok sounds good to me, I told @TransmissionsDev that I'd verify the kovan deployment for him and generally people will find it easier to develop if the contract is verified

@tynes
Copy link
Copy Markdown
Collaborator Author

tynes commented Mar 2, 2021

Once we have hardhat deploy then this script will be deprecated. Just placing it here for now

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants